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Global Variables 
Are Evil! 





“Global variables are responsible for much 
undebuggable code, reentrancy problems, 
global warming, and male pattern baldness. 
Avoid them!” 
— Jack Ganssle © 2021 Philip Koopman 1 
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Global Variables Are Evil! a st 


= Anti-Patterns: 
e More than a few read/write globals 
e Globals shared between tasks/threads 
e Variables have larger scope than needed 


= Global variables are visible everywhere: 
e Use of globals indicates poor modularity Ms 
— Globals are prone to tricky bugs and race conditions 
e Local static variables are best if you need persistence 
— File static variables can be OK if used properly 
— Dont make procedures globally visible if not needed 
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Global vs. Static Variables chee, 
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= Globals: 
uint32 t gVar = 0; 
void gProc (...) { 








= File Static: 
static uint32 t fsVar = 0; 
static void fsProc (...) 1), ciel 
e Only inside .c file : 
e Use with small .c files & ul ws, ees 
e Like C++ “private” : 











= Global risks 


e Written from anywhere 
— Debugging: who wrote it? 





https://goo.gl/PhhDcY 





e Read from anywhere = Local Static: 
— Changes break everything void gProc (...) 
Multithreaded race conditions { static uint32 t sVar = 0; 
Increased complexity ug 





— Data flow “spaghetti” _ e Persistent variable value oN 


e Cant be seen outside procedure 
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Avoiding And Removing Globals Mellon 


ellon 
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= Define smallest scope possible (variables and procedures) 
e Change global to file static; file static to local static —=3 


m Arrange .c files based on access to data 
e Example: time of day updated by ISR (mar, & 
— File static time of day variable in TimeOfDay.c Al oP Ze: 
— Put timer tick ISR in TimeOfDay.c £ 
— Put procedure to disable interrupts & read time of day in TimeOfDay.c 
= Configuration values & constants 
e Use const keyword — prevents multiple writers 
e Read-only access to global configuration data structure 
e Limit visibility to need-to-know within relevant .h file 
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=m Use smallest practical scope for variables & procedures 
e Ideally, zero global variables 
e Use file static if you must; local static if you can 
e A good compiler will generate efficient code 





m Reorganize code to reduce scope 


e Write anything except locking variables only in one place 
e File static variables for small groups of functions 
— More or less the idea of C++ private keyword 
— Take care of data locking when reading 


= Global Variable Pitfalls 


e Lots of global variables is a sign of bad code 
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https://betterembsw.blogspot.com/2013/09/getting-rid-of-global-variables.html 
You have a "globals.c’ file that defines a mess of globals, including: 


int g_ ErrCount; 


which might be used to tally the number of run-time errors seen by the system. I've 
used ag _" naming convention to emphasize that is a global, which means that every .c 
file in the program can read and write this variable with wild abandon. 


Let's say you also have the following places this variable is referenced, including 
globals.c just mentioned: 


globals.c: 
globals.h: 
Init.c: 


moduleX.c: 
moduleY.c: 
moduleZ.c: 


int g_ ErrCount; // define the variable 

extern int g ErrCount; // other files include this 

g_ ErrCount = 0; // init when program starts 

g ErrCount++; // tally another error 

XVar = g_ErrCount; // get current number of errors 

g_ ErrCount = 0; // clear number of reported errors — ©2021 Philip Koopman 6 
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Create an Error Counting Module eee: 
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Create separate “object” for error counting: ErrCount.c 
globals.c: //not needed any more for this variable 
ErrCount.c: intg ErrCount; // define the variable 
ErrCount.h: extern int g ErrCount; // other files include this 
init.c: g_ ErrCount = 0; // init when program starts 
moduleX.c: g ErrCount++; // tally another error 

moduleY.c: XVar=g_ErrCount; // get current number of errors 
moduleZ.c: g ErrCount = 0; // clear number of reported errors 
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Initialize Where Defined pa 
ErrCount.c: intg ErrCount=0; // define and init variable 
ErrCount.h: extern int g ErrCount; // other files include this 
init.c: // no longer needed 
moduleX.c: g ErrCount++; // tally another error 
moduleY.c: XVar=g_ErrCount; // get current number of errors 
moduleZ.c: g ErrCount =0; // clear number of reported errors 
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Convert to File Static erty 


= ErrCount.c: static int ErrCount = 0; // only visible in this file 

= ErrCount.h: // static variables are invisible outside .c file 

= moduleX.c: g_ ErrCount++; // tally another error 

=m moduleyY.c: XVar=g_ErrCount; // get current number of errors 
=m moduleZ.c: g ErrCount=0; // clear number of reported errors 
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_ Add Accessor Function a, 
ErrCount.c: — static int ErrCount = QO; // only visible in this file 
® inline void ErrCount_Incr() { ErrCount++; } 
@ inline int ErrCount_Get() { return(ErrCount); } 
® inline void ErrCount_Reset() { ErrCount = 0; } 
ErrCount.h: 
° inline void ErrCount_Incr(); // increment the count 
8 inline int ErrCount_Get(); // get current count value 
© inline void ErrCount_Reset(); // reset count 
s // Note that there is NO access to ErrCount directly 


moduleX.c: ErrCount Incr(); // tally another error 
moduleY.c: XVar=ErrCount Get(); // get current number of errors 
moduleZ.c: ErrCount_Reset(); // clear number of reported errors 
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Advantages of this Approach Tae, 


software authors can only perform intended functions specific to an error counter: increment, 
read, and reset. Setting to an arbitrary value isn't allowed. If you don't want the value changed 
other than via incrementing, you can just delete the reset function. This prevents some types of 
bugs from ever happening. 


If you need to change the data type or representation of the counter used that all happens 
inside ErrCount.c with no effect on the rest of the code. For example, if you find a bug with error 
counts overflowing, it is a lot easier to fix that in one place than every place that increments the 
counter! 


lf you are debugging with a breakpoint debugger it is easier to Know when the variable has 
been modified, because you can get rid of the "inline" Keywords and put a breakpoint in the 
access functions. Otherwise, you need watchpoints, which aren't always available. 


If different tasks in a multitasking system need to access the variable, then it is a lot easier to 
get the concurrency management right inside a few access functions than to remember to get it 
right everywhere the variable is read or written (get it right once, use those functions over and 
over). Don't forget to make the variable volatile and disable interrupts when accessing it if 
concurrency is an issue. 
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https://xked.com/2309/ 


